Summary

The chat() method in src/opengradient/client/llm.py used a mutable default argument:

tools: Optional[List[Dict]] = []

This is a known Python anti-pattern that can cause shared state across function calls, leading to subtle and hard-to-debug issues.

Fix

Changed to the safe pattern:

tools: Optional[List[Dict]] = None

The existing code already handles None correctly since it uses if tools: which evaluates to False for both [] and None.

Related

Fixes item #5 from issue #157.